Skip to content

Conversation

@edsoncelio
Copy link
Contributor

Describe your changes

Remove localization maintainer (edsoncelio)

Related issue number or link (ex: resolves #issue-number)

N/A

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • [ x I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@netlify
Copy link

netlify bot commented Apr 13, 2025

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit c8e6c06
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/67fcf4b84cc73400083bd152
😎 Deploy Preview https://deploy-preview-3455--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

I sincerely appreciate your contribution to the Glossary project. I look forward to seeing you again!

Before moving foward with this PR, please create a PR to modify the permissions on cncf/people. Thank you :)

Copy link
Collaborator

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, please fix CODEOWNERS file too

glossary/CODEOWNERS

Lines 58 to 59 in 67d0761

/content/pt-br/ @edsoncelio @brunoguidone @jessicalins @MrErlison
/i18n/pt-br.toml @edsoncelio @brunoguidone @jessicalins @MrErlison

@Okabe-Junya
Copy link
Collaborator

@Okabe-Junya Okabe-Junya added the maintainers Use this label if PR requires maintainers to take action label Apr 14, 2025
Copy link
Collaborator

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for everything @edsoncelio and good luck on your next thing!

@nate-double-u nate-double-u merged commit e7140b1 into main Apr 15, 2025
8 checks passed
@nate-double-u nate-double-u deleted the edsoncelio-patch-1 branch April 15, 2025 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainers Use this label if PR requires maintainers to take action

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants